Skip to content

Conversation

MultisampledNight
Copy link
Contributor

Closes #121. Somehow NotNan doesn't impl Float and as such it doesn't like my HasKernel impl, so I left that out for now until I have time to actually properly look at this.

@MultisampledNight
Copy link
Contributor Author

Tests fail due to a rename of clippy lints, not my changes. Should include a rename either way?

@mbrubeck
Copy link
Collaborator

mbrubeck commented Mar 9, 2023

Yes, feel free to update that lint in this PR (or as a separate PR if you prefer).

@MultisampledNight
Copy link
Contributor Author

MultisampledNight commented Mar 9, 2023

Technically this is ready and okay so far, but according to the geo maintainers on Discord they want to change their RobustKernel to remove the Float requirement, so this way the vendored NotNanRobustKernel wouldn't be required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OrderedFloat and NotNan should implement geo::HasKernel (behind feature)
2 participants